Comments

HW#66

Open

yaAppolitta wants to merge 12 commits intoYandex-Practicum:master from

yaAppolitta:master

Open

HW #66
yaAppolitta wants to merge 12 commits intoYandex-Practicum:master from
yaAppolitta:master

Conversation

@yaAppolitta

Copy link

@yaAppolitta yaAppolitta commented Oct 4, 2022 •

edited

Loading

Спринт 2. Проектная работа 1.
попытка 2

s-buvaka and others added 12 commits

July 27, 2022 22:57


 Init commit

83a7216

@yaAppolitta


 Change project name

@yaAppolitta


 Merge remote-tracking branch 'origin/master'

83e4746

@yaAppolitta


 Merge remote-tracking branch 'origin/master'

ba761a0

@yaAppolitta


 Спринт 2 . Проектная работа 1.

544d467

Финал. Попытка 1

@yaAppolitta


 Merge remote-tracking branch 'origin/master'

9e32c9e

@yaAppolitta


 Спринт 2 . Проектная работа 1.

27c3c49

Финал. Попытка 1

@yaAppolitta


 Спринт 2 . Проектная работа 1.

df34b55

Финал. Попытка 1

@yaAppolitta


 Спринт 2 . Проектная работа 1.

bb64e7e

Финал. Попытка 1

@yaAppolitta


 Спринт 2 . Проектная работа 1.

fab258e

Финал. Попытка 1

@yaAppolitta


 Спринт 2 . Проектная работа 1.

52ba7b8

Финал. Попытка 1

@yaAppolitta


 Спринт 2 . Проектная работа 1.

d75519a

Финал. Попытка 2

MagicUnderHood

MagicUnderHood reviewed

Oct 6, 2022

View reviewed changes

src/main/java/Main.java

visitor = scanner.nextInt();

if (visitor <=1) {

System.out.println("Некорректное кол-во гостей");

System.out.println("Введите колличество гостей");

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022 •

edited

Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно перенести эту строчку наверх, сразу после while, тогда отсюда и перед while её можно удалить

src/main/java/Main.java

Product list = new Product();

Calculate menu_list = new Calculate();

while (enough) {

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно просто прописать while(true)

src/main/java/Main.java

boolean enough = true;

Product list = new Product();

Calculate menu_list = new Calculate();

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для именования переменных лучше использовать верблюжий регистр, то есть menuList

src/main/java/Main.java

menu_list.getListPrice(list.price);

System.out.println("Хотите ввести ещё товар?");

System.out.println("Eсли вы хотите ввести ещё товар, введите ДА");

System.out.println("Eсли вы хотите ввести ещё товар, введите Завершить ");

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Наверное, тут должно быть "Если вы не хотите больше вводить товары, введите Завершить"

src/main/java/Main.java

}

if (!enough){

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот if можно убрать, и переменную enough тоже. Из цикла while может быть выход, только если пользователь введет Завершить, и тогда должен будет выполниться код здесь

src/main/java/Main.java

if ((rub == 0 ) || (rub == 5) || (rub == 6) || (rub == 7 ) || (rub == 8 ) || (rub == 9)){

rubString = "рублей";

} else if ((rub == 2) || (rub == 4) || (rub == 3) ){

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно упростить - rub >= 2 && rub <= 4

src/main/java/Main.java

String receipt = "";

double pay = 0;

public void getAddList (String name, double price) {

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Этот и getListPrice методы называются "get", то есть "получить", но ничего не возвращают. Лучше назвать "addProduct" и "addPrice". Ещё можно передавать в параметре этого метода сразу объект класса Product, и из него брать имя и цену

src/main/java/Main.java

}

public static class Product {

Copy link

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Классы лучше в отдельный файл располагать, и тогда не надо будет делать их статическими

src/main/java/Main.java

String rubString = "";

int rub = (int) (pay % 10);

int rubTeen = (int) (pay % 100);

if ((rub == 0 ) || (rub == 5) || (rub == 6) || (rub == 7 ) || (rub == 8 ) || (rub == 9 ) || ((rubTeen >= 11) && (rubTeen <= 14))){

Copy link

@MagicUnderHood MagicUnderHood Oct 6, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно упростить условие - rub == 0 || (rub >= 5 && rub <= 9) || (rubTeen >= 11 && rubTeen <= 14)

Labels

None yet

2 participants

@yaAppolitta @MagicUnderHood

Comments

Conversation

@yaAppolitta yaAppolitta commented Oct 4, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 5, 2022

Choose a reason for hiding this comment

Uh oh!

@MagicUnderHood MagicUnderHood Oct 6, 2022

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

@yaAppolitta yaAppolitta commented Oct 4, 2022 •

edited

Loading

@MagicUnderHood MagicUnderHood Oct 5, 2022 •

edited

Loading